Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

modulesync 4.0.0 #334

Merged
merged 7 commits into from
Mar 1, 2021
Merged

modulesync 4.0.0 #334

merged 7 commits into from
Mar 1, 2021

Conversation

bastelfreak
Copy link
Member

modulesync 4.0.0

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EL6 failed:

       	**********     Wrong JVM version! You are running with java version "1.7.0_261" but JIRA requires at least 1.8 to run.      **********

Previously it didn't run and I'm not sure if Java 8 is available on EL6. Since it's going EOL I'm recommending to drop EL6.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should also drop .sync.yml which is now redundant.

@ekohl
Copy link
Member

ekohl commented Dec 29, 2020

Not sure why PostgreSQL doesn't start on Ubuntu 16.04. It may be due to PIDFIle. #326 also broke Ubuntu due to bashisms. The next failure on Ubuntu 18.04 is that it doesn't recognize openjdk 11. Looks like the default version doesn't support that but recent ones do.

For now I'm giving up on this and someone who actually uses the module needs to step in.

@kenyon kenyon force-pushed the modulesync branch 2 times, most recently from e658ae7 to 924de44 Compare February 14, 2021 06:31
@kenyon
Copy link
Member

kenyon commented Feb 14, 2021

Not sure why PostgreSQL doesn't start on Ubuntu 16.04. It may be due to PIDFIle. #326 also broke Ubuntu due to bashisms. The next failure on Ubuntu 18.04 is that it doesn't recognize openjdk 11. Looks like the default version doesn't support that but recent ones do.

For now I'm giving up on this and someone who actually uses the module needs to step in.

This is the only remaining issue preventing the tests from passing. PostgreSQL on Ubuntu 16.04 starts but systemd hangs at "activating":

root@ubuntu1604-64-1:~# systemctl status postgresql@9.5-main
● postgresql@9.5-main.service - PostgreSQL Cluster 9.5-main
   Loaded: loaded (/lib/systemd/system/postgresql@.service; disabled; vendor preset: enabled)
   Active: activating (start) since Sun 2021-02-14 07:56:53 UTC; 37s ago
  Process: 6281 ExecStart=/usr/bin/pg_ctlcluster --skip-systemctl-redirect %i start (code=exited, status=0/SUCCESS)
   CGroup: /docker/378b796a749da8c903ba5f6f2da85ecf2ebd218038129f71e9f39a1c084d1d9a/system.slice/system-postgresql.slice/postgresql@9.5-main.service
           ├─6286 /usr/lib/postgresql/9.5/bin/postgres -D /var/lib/postgresql/9.5/main -c config_file=/etc/postgresql/9.5/main/postgresql.conf
           ├─6288 postgres: 9.5/main: checkpointer process
           ├─6289 postgres: 9.5/main: writer process
           ├─6290 postgres: 9.5/main: wal writer process
           ├─6291 postgres: 9.5/main: autovacuum launcher process
           └─6292 postgres: 9.5/main: stats collector process

Feb 14 07:56:53 ubuntu1604-64-1 systemd[1]: Starting PostgreSQL Cluster 9.5-main...
Feb 14 07:56:55 ubuntu1604-64-1 systemd[1]: postgresql@9.5-main.service: New main PID 6286 does not belong to service, and PID file is not owned by root. Refusing.
Feb 14 07:56:55 ubuntu1604-64-1 systemd[1]: postgresql@9.5-main.service: New main PID 6286 does not belong to service, and PID file is not owned by root. Refusing.

root@ubuntu1604-64-1:~# ll /run/postgresql/9.5-main.pid
-rw-r--r-- 1 postgres postgres 5 Feb 14 07:56 /run/postgresql/9.5-main.pid

root@ubuntu1604-64-1:~# cat /run/postgresql/9.5-main.pid
6286

🤷

Seems to be related to these:

Same problem here: voxpupuli/puppet-puppetboard#292

@ekohl
Copy link
Member

ekohl commented Feb 14, 2021

We have a pidfile workaround in our templates which uses an older version of Ubuntu 16.04 but I'd prefer to drop Ubuntu 16.04. It's going EOL soon anyway.

kenyon added a commit that referenced this pull request Feb 15, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
kenyon added a commit that referenced this pull request Feb 15, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
ekohl pushed a commit that referenced this pull request Feb 22, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
@ekohl ekohl removed the tests-fail label Feb 22, 2021
ekohl pushed a commit that referenced this pull request Feb 23, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
ekohl pushed a commit that referenced this pull request Feb 23, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
@ekohl
Copy link
Member

ekohl commented Feb 23, 2021

This is green, but care should be taken. #345 is included and I've included an alternative to #341 as well.

Copy link
Member Author

@bastelfreak bastelfreak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't approve it, but looks good to me.

@ekohl
Copy link
Member

ekohl commented Feb 23, 2021

@kenyon thoughts?

ekohl and others added 7 commits February 28, 2021 22:29
By specifying the dependencies on the resources, the chain is simpler.
This allows Java to be installed even if PostgreSQL fails. The benefit
is that it's easier to see in one go if things fail.
1112c0a changed the file to use
bashisms, but on Ubuntu /bin/sh is dash, not bash. [[ is not supported.
These JVM args are unrecognized on CentOS 7 and cause Jira to fail to
start.
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
#334 (comment)
for details.

Also update some versions in the README.
@kenyon
Copy link
Member

kenyon commented Mar 1, 2021

Looks good to me. Rebased on current master. Updated the Jira version in the alternative commit to #341.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get this in. I opened #346 for the Ubuntu 16.04 -> 18.04 change for the changelog.

@ekohl ekohl merged commit 73eae4e into master Mar 1, 2021
@ekohl ekohl deleted the modulesync branch March 1, 2021 09:49
cegeka-jenkins pushed a commit to cegeka/puppet-jira that referenced this pull request Oct 13, 2021
Ubuntu 16.04 standard support ends in a few weeks. Including Ubuntu
16.04 support causes our acceptance tests to hang. See
voxpupuli#334 (comment)
for details.

Also update some versions in the README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop Ubuntu 16.04, Add 18.04 instead
3 participants